Skip to content

Comments

[3006.x] Add coverage for salt/crypt.py#65417

Open
MKLeb wants to merge 1 commit intosaltstack:3006.xfrom
MKLeb:coverage/3006.x/batch-7/crypt
Open

[3006.x] Add coverage for salt/crypt.py#65417
MKLeb wants to merge 1 commit intosaltstack:3006.xfrom
MKLeb:coverage/3006.x/batch-7/crypt

Conversation

@MKLeb
Copy link
Contributor

@MKLeb MKLeb commented Oct 17, 2023

What does this PR do?

See title

What issues does this PR fix or reference?

Fixes: One file for #64739

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

@MKLeb MKLeb requested a review from a team as a code owner October 17, 2023 20:01
@MKLeb MKLeb requested review from Ch3LL and removed request for a team October 17, 2023 20:01
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 21:05 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 21:06 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 21:07 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 21:07 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 21:08 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 21:08 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 23:42 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 23:43 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 23:43 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 23:43 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 23:43 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 17, 2023 23:43 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 01:07 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 01:07 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 01:07 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 01:07 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 01:07 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 01:07 — with GitHub Actions Inactive
@MKLeb MKLeb changed the title [3006.x] Add coverage for salt/crypt.py [3006.x] Add coverage for salt/crypt.py Oct 18, 2023
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 05:21 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 05:22 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 05:23 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 05:24 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 05:24 — with GitHub Actions Inactive
@MKLeb MKLeb temporarily deployed to ci October 18, 2023 05:25 — with GitHub Actions Inactive
@dwoz dwoz modified the milestones: Sulfur v3006.7, Sulfur v3006.9 Jun 22, 2024
@dwoz dwoz requested a review from a team as a code owner March 20, 2025 21:24
@dmurphy18
Copy link
Contributor

@MKLeb Need conflicts resolved and what about @s0undt3ch comments

@dmurphy18 dmurphy18 requested review from dmurphy18 and removed request for Ch3LL March 20, 2025 21:28
@dmurphy18
Copy link
Contributor

@dwoz Do we care about crypt ?

@dwoz dwoz modified the milestones: Sulfur v3006.12, Sulfer v3006.13 Jun 13, 2025
@twangboy twangboy added test:full Run the full test suite pending-changes The pull request needs additional changes before it can be merged and removed test:coverage labels Jul 2, 2025
@twangboy twangboy modified the milestones: Sulfur v3006.18, 3006.20 Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending-changes The pull request needs additional changes before it can be merged test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants